Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make Embedded Object Fields hide-able #68

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

KINKCreative
Copy link

This adds 2 simple config params to hide the description field (hide_description) and the other fields (hide_fields). Pretty handy - for me.

@nyeholt
Copy link
Collaborator

nyeholt commented Nov 8, 2016

More correct would be to instead add an extension point, and in your extension remove all items from the passed in $fields list

@KINKCreative
Copy link
Author

You're right - $this->extend(...) right? I haven't done this before :)

Sent from my iPhone

On Nov 8, 2016, at 1:48 AM, Marcus [email protected] wrote:

More correct would be to instead add an extension point, and in your extension remove all items from the passed in $fields list


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

@sheadawson
Copy link
Owner

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants